Skip to content

[19.0][ADD] ai_tool, ai_connection - #100

Closed
volkantasci wants to merge 2 commits into
OCA:19.0from
volkantasci:19.0-add-ai-connection
Closed

[19.0][ADD] ai_tool, ai_connection#100
volkantasci wants to merge 2 commits into
OCA:19.0from
volkantasci:19.0-add-ai-connection

Conversation

@volkantasci

Copy link
Copy Markdown

Faithful port of ai_tool and ai_connection from 18.0 to 19.0.

Changes vs the 18.0 sources (required for Odoo 19):

  • attrsetter now imported from odoo.orm.decorators (ai_tool/tools.py)
  • ai_connection tests migrated from odoo_test_helper.FakeModelLoader (Odoo 18 only) to the native odoo.orm.model_classes.add_to_registry API
  • manifest versions bumped to 19.0.1.0.0

ai_connection depends on ai_tool, so both are in one PR. This connection layer is the dependency for the upcoming ai_document_extraction module.

@volkantasci

Copy link
Copy Markdown
Author

Hi @etobella 👋

This is the base PR for #96 and #106 (ai_tool + ai_connection port to 19.0). All CI is green (pre-commit, Odoo/OCB, codecov) and it is required before the other two can be rebased cleanly.

When you have a moment, could you take a look? Happy to adjust anything.

Thanks for your time!

@etobella

Copy link
Copy Markdown
Member

Migration is not done following OCA guidelines.

@volkantasci

Copy link
Copy Markdown
Author

Restructuring this PR per @etobella's review that the migration was not done following OCA guidelines (Migration to version 19.0):

  • the module histories from 18.0 are now replayed (git format-patch | git am -3 --keep) instead of re-adding files, preserving original authors, Weblate translations and bot commits
  • proper [MIG] <module>: Migration to 19.0 commits and per-module PRs

Replaced by:

The final trees are byte-identical to the content that passed CI here.

@volkantasci volkantasci closed this Sep 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:ai_connection Module ai_connection mod:ai_tool Module ai_tool series:19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants